-
Notifications
You must be signed in to change notification settings - Fork 929
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix a logic bug #1634
Merged
Merged
fix a logic bug #1634
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Lvnszn
approved these changes
Dec 2, 2021
PhilYue
approved these changes
Dec 2, 2021
Mulavar
approved these changes
Dec 2, 2021
cjphaha
approved these changes
Dec 2, 2021
LGTM👍 |
AlexStocks
approved these changes
Dec 3, 2021
AlexStocks
added a commit
that referenced
this pull request
Dec 3, 2021
This reverts commit 5604ba5.
AlexStocks
added a commit
that referenced
this pull request
Dec 3, 2021
AlexStocks
added a commit
that referenced
this pull request
Dec 21, 2021
* Merge 3.0 to master (#1606) * build(deps): bump github.com/hashicorp/vault/sdk from 0.2.1 to 0.3.0 (#1552) Bumps [github.com/hashicorp/vault/sdk](https://github.com/hashicorp/vault) from 0.2.1 to 0.3.0. - [Release notes](https://github.com/hashicorp/vault/releases) - [Changelog](https://github.com/hashicorp/vault/blob/main/CHANGELOG.md) - [Commits](hashicorp/vault@sdk/v0.2.1...v0.3.0) --- updated-dependencies: - dependency-name: github.com/hashicorp/vault/sdk dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * delete useless codes (#1551) * Update CHANGELOG.md * [#1559]: adjust the startup process of the configuration center (#1560) * [#1559]: 1. fix the startup process of the configuration center(remove error log) 2. refactor zk registry listener * delete redundant dynamic configuration center cache Co-authored-by: dongjianhui03 <dongjianhui03@meituan.com> * [#1276]modify the name of the receiver to a single character (#1561) Co-authored-by: dongjianhui03 <dongjianhui03@meituan.com> * style(common): rename constants in camel-case style (#1563) * style(common): rename constants in camel-case style * style(common): rename Ttl to TTL * fix: #1558: Set root config to global ptr in Init() function. (#1564) * fix: #1558 * fix: change the api to start the app by config-api * fix: in the Consumer service, the Reference config property ‘check’ use the Consumer config when omitted * Triple upgrade to grpc (#1566) * fix: init triple grpc * fix: delete replace * Fix: upgrade triple grpc * Fix: remove unused file * Fix decrease grpc go version to 1.14 * Fix: add server side serialization * register not use metadata configuration. * build(deps): bump github.com/go-resty/resty/v2 from 2.3.0 to 2.7.0 (#1579) Bumps [github.com/go-resty/resty/v2](https://github.com/go-resty/resty) from 2.3.0 to 2.7.0. - [Release notes](https://github.com/go-resty/resty/releases) - [Commits](go-resty/resty@v2.3.0...v2.7.0) --- updated-dependencies: - dependency-name: github.com/go-resty/resty/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump google.golang.org/grpc from 1.41.0 to 1.42.0 (#1578) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.41.0 to 1.42.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.41.0...v1.42.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github.com/knadh/koanf from 1.3.0 to 1.3.2 (#1577) Bumps [github.com/knadh/koanf](https://github.com/knadh/koanf) from 1.3.0 to 1.3.2. - [Release notes](https://github.com/knadh/koanf/releases) - [Commits](knadh/koanf@v1.3.0...v1.3.2) --- updated-dependencies: - dependency-name: github.com/knadh/koanf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: registry param * fix: fix ns and group * fix: integrate test * Fix: integrate test * fix: url fmt * fix: nacos servicediscovery group * Fix: enhance zk * start application (#1571) Co-authored-by: wangxiaowei14227 <wangxiaowei14227@autohome.com.cn> Co-authored-by: lizhixin.lzx <lizhixin.lzx@alibaba-inc.com> Co-authored-by: LaurenceLiZhixin <382673304@qq.com> * graceful shutdwon filter (#1585) Co-authored-by: wangxiaowei14227 <wangxiaowei14227@autohome.com.cn> * Fix: some bugs and features for 3.0 (#1586) * fix: some bugs * Fix: remove file\k8s registry, k8s remote * refactor listenDirEvent 1. remove CLEAR label; 2. remove register/unregister logic for watch children; 3. reorder the zk dynamic configuration code 4. adapt the gost * hessian encode attachments return error (#1588) * upgrade gost dependency version * upgrade hessian to v1.9.5 (#1595) * build(deps): bump k8s.io/apimachinery from 0.16.9 to 0.22.4 (#1600) Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.16.9 to 0.22.4. - [Release notes](https://github.com/kubernetes/apimachinery/releases) - [Commits](kubernetes/apimachinery@v0.16.9...v0.22.4) --- updated-dependencies: - dependency-name: k8s.io/apimachinery dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * ftr: triple tracing (#1596) * ftr: triple tracing * fix: if-else to switch * [fix #1590] filter single instance (#1591) * filter single instance * filter single instance Co-authored-by: wangxiaowei14227 <wangxiaowei14227@autohome.com.cn> * Ftr/triple reflect support (#1603) * ftr: triple-reflection * fix: comment and var name * Fix: comment typo * Fix: official website Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com> Co-authored-by: Mulavar <978007503@qq.com> Co-authored-by: dongjianhui03 <dongjianhui03@meituan.com> Co-authored-by: Xuewei Niu <a@nxw.name> Co-authored-by: PhilYue <phil.yue@foxmail.com> Co-authored-by: lizhixin.lzx <lizhixin.lzx@alibaba-inc.com> Co-authored-by: sanxun0325 <bbz17640380550@163.com> Co-authored-by: Xuewei Niu <justxuewei@apache.org> Co-authored-by: zhaoyunxing <zhaoyunxing@apache.org> Co-authored-by: wangxw <2484713618@qq.com> Co-authored-by: wangxiaowei14227 <wangxiaowei14227@autohome.com.cn> Co-authored-by: 望哥 <gelnyang@163.com> * fix unexported bug (#1634) * Revert "fix unexported bug (#1634)" (#1635) This reverts commit 5604ba5. * add OPay logo to user list Co-authored-by: Laurence <45508533+LaurenceLiZhixin@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com> Co-authored-by: Mulavar <978007503@qq.com> Co-authored-by: dongjianhui03 <dongjianhui03@meituan.com> Co-authored-by: Xuewei Niu <a@nxw.name> Co-authored-by: PhilYue <phil.yue@foxmail.com> Co-authored-by: lizhixin.lzx <lizhixin.lzx@alibaba-inc.com> Co-authored-by: sanxun0325 <bbz17640380550@163.com> Co-authored-by: Xuewei Niu <justxuewei@apache.org> Co-authored-by: zhaoyunxing <zhaoyunxing@apache.org> Co-authored-by: wangxw <2484713618@qq.com> Co-authored-by: wangxiaowei14227 <wangxiaowei14227@autohome.com.cn> Co-authored-by: 望哥 <gelnyang@163.com> Co-authored-by: 刘月财 <38887641+luky116@users.noreply.github.com> Co-authored-by: AlexStocks <alexstocks@foxmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix a logic bug